Summary of Changes

Hello, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request addresses a compilation failure in the coverage CI job by adjusting how Spec structs are instantiated within benchmark code. The change ensures compatibility with the #[non_exhaustive] attribute, allowing benchmarks to compile successfully without altering the struct's definition.

Highlights

• CI Fix for non_exhaustive struct: Resolved a E0639 compilation error in the coverage CI job, specifically within lib/benches/parse.rs, which occurred because the #[non_exhaustive] attribute on the Spec struct prevented direct struct literal construction in benchmark crates.
• Spec struct initialization: Modified the build_small_spec and build_large_spec functions to initialize Spec structs using Spec::default() followed by field-by-field assignment, replacing the previous struct literal syntax.

Greptile Summary

This PR fixes a compilation error in lib/benches/parse.rs caused by Spec being annotated with #[non_exhaustive]. Because benchmarks are compiled as a separate crate, the previous struct-literal syntax (even with ..Default::default()) violated Rust's E0639 rule that prohibits non-exhaustive struct construction outside the defining crate.

• Fix in build_small_spec: Replaced struct literal with Spec::default() + field assignments for name, bin, and cmd.
• Fix in build_large_spec: Same pattern applied identically.
• Both changes are mechanically correct and produce semantically equivalent results — all unset fields still receive their Default values as before.

Confidence Score: 5/5

• This PR is safe to merge — it is a minimal, correct fix for a compile error with no logic changes.
• The change is purely mechanical: it swaps a forbidden struct-literal form for the idiomatic Default + field-assignment pattern required when a type is #[non_exhaustive]. No logic is altered, all field values remain identical, and Spec derives Default so Spec::default() is guaranteed to work. There are no new dependencies, no behavioural changes, and the fix directly addresses the stated E0639 compiler error.
• No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["build_small_spec() / build_large_spec()"] --> B["Spec::default()"]
    B --> C["spec.name = ..."]
    C --> D["spec.bin = ..."]
    D --> E["spec.cmd = cmd"]
    E --> F["return spec"]

    style B fill:#c8f7c5,stroke:#27ae60
    style A fill:#f9f9f9,stroke:#999

_{Reviews (1): Last reviewed commit: "fix: use field assignment instead of str..." | Re-trigger Greptile}

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.40%. Comparing base (84a198c) to head (c3424dc).
⚠️ Report is 7 commits behind head on main.

@@            Coverage Diff             @@
##             main     #552      +/-   ##
==========================================
- Coverage   77.94%   72.40%   -5.54%     
==========================================
  Files          48       48              
  Lines        6682     6830     +148     
  Branches     6682     6830     +148     
==========================================
- Hits         5208     4945     -263     
- Misses       1114     1242     +128     
- Partials      360      643     +283     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.